• Move check-in logic from CheckInModal to Header parent component
• Add isCheckedIn prop to CheckInModal for cleaner state management
• Auto-open check-in modal on first login if not checked in
• Update FallingIcons styling from background to border
• Format and sitemap updates

Move check-in logic to Header parent component

• Import format from date-fns for date formatting
• Calculate today and isCheckedIn in Header component
• Add useEffect to auto-open modal when user is logged in but not checked in
• Pass isCheckedIn prop to CheckInModal component

apps/web/src/Header.tsx

Receive check-in status via prop instead of calculating

• Remove date-fns import and date calculation logic
• Add isCheckedIn prop to component interface
• Replace local date comparison with isCheckedIn prop
• Simplify component by removing date formatting logic

apps/web/src/components/CheckInModal.tsx

Update FallingIcons container styling

• Change background styling from bg-amber-50 to border

apps/web/src/components/FallingIcons.tsx

Description

Header에서 last_check_in이 null인 경우 lastCheckIn을 new Date()로 대체해 isCheckedIn이 true가 되어, 실제로는 미출석인데도 자동
오픈이 되지 않고 모달이 DONE 상태로 초기화됩니다. last_check_in이 null 가능한 타입 정의와 충돌해 신규/초기 사용자에서 출석 기능이 비활성화될 수 있습니다.

Code

apps/web/src/Header.tsx[R61-63]

  const lastCheckIn = user?.last_check_in ?? new Date();
+  const today = format(new Date(), 'yyyy-MM-dd');
+  const isCheckedIn = format(new Date(lastCheckIn), 'yyyy-MM-dd') >= today;

Evidence

User 타입에서 last_check_in은 null 가능인데, Header에서 null을 new Date()로 치환하여(=오늘 날짜) isCheckedIn 비교 결과가 항상
true가 됩니다. 이 값이 CheckInModal의 initalStatus로 전달되어 미출석 사용자도 DONE으로 시작할 수 있습니다.

apps/web/src/types/user.ts[1-7]
apps/web/src/Header.tsx[59-64]
apps/web/src/components/CheckInModal.tsx[16-56]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`last_check_in`이 `null`일 수 있는데도 `?? new Date()`로 오늘 날짜로 치환해버려, 미출석 사용자가 `isCheckedIn=true`로 판정됩니다.

### Issue Context
- `User.last_check_in` 타입은 `Date | null` 입니다.
- 현재 로직은 `null`을 '오늘 출석함'으로 간주합니다.

### Fix Focus Areas
- apps/web/src/Header.tsx[59-64]
- apps/web/src/components/CheckInModal.tsx[16-33]

### Suggested fix direction
- `lastCheckIn`을 `Date | null`로 유지하고, `isCheckedIn` 계산 시 `lastCheckIn`이 없으면 `false`로 처리하세요.
 - 예: `const lastCheckIn = user?.last_check_in ?? null;`
 - 예: `const isCheckedIn = lastCheckIn ? format(new Date(lastCheckIn), 'yyyy-MM-dd') >= today : false;`
- `CheckInModalProps.lastCheckIn`도 `Date | null`로 바꾸고(`useCheckInTimer`는 이미 `Date | null` 허용), 내부 사용처를 이에 맞게 조정하세요.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Description

Header의 자동 오픈 useEffect는 '첫 접속 1회'를 보장하는 가드가 없고, user 쿼리 갱신으로 user 참조가 바뀔 때마다 미출석이면 setOpen(true)를
다시 실행할 수 있습니다. 예를 들어 미출석 상태에서 사용자가 모달을 닫은 뒤 포인트 사용(ThumbUpModal) 등으로 userCheckIn 쿼리가
invalidate/refetch 되면 출석 모달이 다시 강제로 열릴 수 있습니다.

Code

apps/web/src/Header.tsx[R84-88]

+  useEffect(() => {
+    if (!isLoading && !!user && !isCheckedIn) {
+      setOpen(true);
+    }
+  }, [isLoading, user, isCheckedIn]);

Evidence

Header는 (!isLoading && user && !isCheckedIn)이면 무조건 setOpen(true)를 수행하며, 의존성 배열에 user 객체 자체가
포함되어 있어 쿼리 데이터가 갱신될 때마다 effect가 재평가됩니다. 또한 usePatchSetPointMutation 등에서 동일한 ['userCheckIn'] 키를
invalidate하여 refetch를 유발할 수 있고, ThumbUpModal에서 해당 mutation을 실제로 사용합니다.

apps/web/src/Header.tsx[84-88]
apps/web/src/queries/userQuery.ts[5-18]
apps/web/src/queries/userQuery.ts[35-46]
apps/web/src/components/ThumbUpModal.tsx[40-52]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
미출석 상태에서 `userCheckIn` 쿼리가 invalidate/refetch 될 때마다 자동 오픈 effect가 다시 실행되어, 사용자가 모달을 닫아도 출석 모달이 재오픈될 수 있습니다.

### Issue Context
- Header effect는 `user` 객체 참조 변화에 민감합니다.
- `usePatchSetPointMutation`/`usePatchUserCheckInMutation`가 `['userCheckIn']`을 invalidate 합니다.
- ThumbUpModal에서 포인트 차감 mutation이 실행됩니다.

### Fix Focus Areas
- apps/web/src/Header.tsx[84-88]
- apps/web/src/queries/userQuery.ts[21-46]
- apps/web/src/components/ThumbUpModal.tsx[40-52]

### Suggested fix direction
- '첫 접속 1회'를 보장하는 플래그를 추가하세요.
 - 예: `const didAutoOpenRef = useRef(false);` 후 `if (!didAutoOpenRef.current && ...) { setOpen(true); didAutoOpenRef.current = true; }`
 - 또는 사용자가 닫았으면 같은 세션/같은 날짜에는 다시 자동 오픈하지 않도록 `sessionStorage/localStorage`에 기록
- 의존성에서 `user` 객체 전체 대신 `user?.id` 및 `isCheckedIn` 등 안정적인 스칼라 값만 사용해 불필요한 재실행을 줄이세요.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools